Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TracyProfiler] Update to v0.10 #8437

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

[TracyProfiler] Update to v0.10 #8437

wants to merge 4 commits into from

Conversation

IanButterworth
Copy link
Member

@IanButterworth IanButterworth commented Apr 4, 2024

v0.10 released Oct 16, 2023 https://github.com/wolfpld/tracy/releases/tag/v0.10

@topolarity @KristofferC this look good to you?

@IanButterworth IanButterworth changed the title Update TracyProfiler to v0.10 [TracyProfiler] Update to v0.10 Apr 4, 2024
@IanButterworth
Copy link
Member Author

Thanks to @giordano for publishing this pre-release. I tested it and tracy loads, but complains that julia needs to be built against a compatible lib version

Screenshot 2024-04-04 at 11 36 13 AM

@topolarity
Copy link
Contributor

Yeah, unfortunately Tracy is not backwards-compatible so this will need a coordinated bump of LibTracyClient_jll

@giordano
Copy link
Member

giordano commented Apr 4, 2024

Oh, I see TracyProfiler and LibTracyClient technically don't depend on each other, so we can't set compat bounds to avoid these issues. That's annoying.

@topolarity
Copy link
Contributor

Yeah we don't want to make TracyProfiler depend on LibTracyClient since that would cause the client to auto-load.

We could potentially use a dummy package TracyProtocol that they both depend on?

@giordano
Copy link
Member

giordano commented Apr 4, 2024

We could potentially use a dummy package TracyProtocol that they both depend on?

How about the opposite: a simple Julia package which loads both of them, and can set compat bounds. End users would use that one directly, instead of the jlls.

@topolarity
Copy link
Contributor

That would continue to have the problem of loading the client in a process where you only want to run the GUI application though, which is very confusing for users because it means an extra julia appears in menu to connect to and it generally sends no events.

@IanButterworth
Copy link
Member Author

I might be missing something, but you can depend on a package and set compat for it, but not load it. Wouldn't that work?

@giordano
Copy link
Member

giordano commented Apr 4, 2024

In JLLs we don't have "depend on but not load" dependencies, not at the moment.

@IanButterworth
Copy link
Member Author

I've been trying to use the current TracyProfiler_jll on MacOS and it is basically unusable because of the hypersensitive zoom and lack of scrolling via mouse and no scrollbars.

It'd be good to get this in and julia updated.

@topolarity
Copy link
Contributor

Are those issues fixes in 0.10?

@IanButterworth
Copy link
Member Author

I've tried to figure that out from Tracy and CImGUI PRs but haven't managed.

Do you know if I can load this jll with some dummy profile data?

@IanButterworth
Copy link
Member Author

I posted here wolfpld/tracy#631 (comment)

@topolarity
Copy link
Contributor

You can use the sample file used for Tracy's web demo: https://tracy.nereid.pl/Tracy-release.data

Despite the file extension, it's really a .tracy file

@IanButterworth
Copy link
Member Author

Ah good tip.

No luck, unfortunately.

Screenshot 2024-07-14 at 7 33 08 PM

I'll post back on the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants